Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure uncaught throwables in task fibers are visible to the user #397

Merged
merged 2 commits into from
Apr 3, 2024

Conversation

s-ludwig
Copy link
Member

@s-ludwig s-ludwig commented Apr 1, 2024

Prefers logFatal over stderr.writeln so that the error also ends up in log files that may be configured. Also displays a message box for subsystem "windows" executables that don't have a console attached.

Prefers logFatal over stderr.writeln so that the error also ends up in log files that may be configured. Also displays a message box for subsystem "windows" executables that don't have a console attached.
@s-ludwig s-ludwig force-pushed the better_fatal_task_error_logging branch from f13096f to be69b76 Compare April 2, 2024 06:58
@s-ludwig
Copy link
Member Author

s-ludwig commented Apr 2, 2024

The MessageBox approach probably needs some refinement prior to merge, as this would not interact well when a subsystem:windows application is run as a non-interactive service.

@s-ludwig s-ludwig requested a review from l-kramer April 2, 2024 15:15
@l-kramer l-kramer merged commit b2462e4 into master Apr 3, 2024
12 checks passed
@l-kramer l-kramer deleted the better_fatal_task_error_logging branch April 3, 2024 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants